-
Notifications
You must be signed in to change notification settings - Fork 158
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add possibility to route requests to sidecar container #270
Conversation
@pierluigilenoci since you are the most active committer of this repo, may we kindly ask you if you could please take a look? Although quite simple, this fix would be very beneficial for us 🙂 thank you so much in advance! |
@edorizzardi Please update the message for ArtifactHub and rebase the PR from the main branch. 🙏🏻 👍🏻 |
@pierluigilenoci thank you. Done |
@edorizzardi The chart bump is now missing (this is a new feature, so it must be a "minor"). ➕ |
@pierluigilenoci Updated minor version. Plus I added the parameter in the list, and added a small paragraph on how to use my changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are still some details to iron out, sorry if I'm being pedantic. 🙏🏻
Description
This PR changes the
Service
such that it allows to route requests to an arbitrary port. This is not a breaking change.